Skip to content

fix(ci): migrate self-hosted to ubuntu-latest for PR safety (ADR-FM-027 D3) - #91

Merged
hummbl-agent merged 1 commit into
mainfrom
fix/devin/ci-ubuntu-latest
Aug 3, 2026
Merged

fix(ci): migrate self-hosted to ubuntu-latest for PR safety (ADR-FM-027 D3)#91
hummbl-agent merged 1 commit into
mainfrom
fix/devin/ci-ubuntu-latest

Conversation

@hummbl-dev

Copy link
Copy Markdown
Collaborator

Summary

  • Migrates runs-on from [self-hosted, Linux, X64, wsl, <repo>] to ubuntu-latest
  • Eliminates P1 security exposure: pull_request trigger was running untrusted PR code on self-hosted runners
  • Per GitHub guidance, self-hosted runners should never be used on public repos with PR triggers
  • ubuntu-latest is ephemeral (destroyed after each job), eliminating the risk

Context

ADR-FM-027 D3 Phase 1 — Fleet CI runner security remediation. 6 public repos under hummbl-dev-org had pull_request + self-hosted in ci.yml. This PR fixes one of them.

Test plan

  • CI passes on this PR (runs on ubuntu-latest, confirming the migration works)
  • No test failures related to runner environment change

…27 D3)

Eliminates P1 security exposure: pull_request trigger was running untrusted PR code on self-hosted runners. Per GitHub guidance, self-hosted runners should never be used on public repos with PR triggers. ubuntu-latest is ephemeral and eliminates the risk.

@hummbl-agent hummbl-agent left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Operator-directed batch triage merge.

@hummbl-agent
hummbl-agent enabled auto-merge (squash) August 2, 2026 05:30
@hummbl-agent
hummbl-agent merged commit 8b2205f into main Aug 3, 2026
2 of 6 checks passed
@hummbl-agent
hummbl-agent deleted the fix/devin/ci-ubuntu-latest branch August 3, 2026 16:15
hummbl-agent pushed a commit that referenced this pull request Aug 3, 2026
Public repos use ubuntu-latest. Replace [self-hosted, Linux, X64,
wsl, base120] with ubuntu-latest across 4 workflow files (base120,
ci, guardrails, mirror-conformance). Unblocks PRs #90 and #91
which were failing with no runner registered for the base120 label.

Generated with [Devin](https://devin.ai)

Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>
hummbl-agent added a commit that referenced this pull request Aug 3, 2026
Public repos use ubuntu-latest. Replace [self-hosted, Linux, X64,
wsl, base120] with ubuntu-latest across 4 workflow files (base120,
ci, guardrails, mirror-conformance). Unblocks PRs #90 and #91
which were failing with no runner registered for the base120 label.

Generated with [Devin](https://devin.ai)

Co-authored-by: Reuben Bowlby <reuben@hummbl.io>
Co-authored-by: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>
@hummbl-agent

Copy link
Copy Markdown
Contributor

CI runner migration merged in #92. Re-triggering CI checks against updated main.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants